…ConfigurationService.setDefaultMarkingDefinitionsForObject

- I believe this was the source of the circular dependencies issue. There are no more circuluar imports between the AttackObjectsService, RelationshipsService, IdentitiesService, UserAccountsService, and SystemsConfigurationService.
- This method makes more sense in the systems-configuration-service module anyways, considering other similar functions are implemented there.
- Had to rename the function because an identically named function that processes STIX IDs instead of STIX objects already exists in the system-configuration-service module. We may consolidate these in the future.

…enter-for-threat-informed-defense/attack-workbench-rest-api into refactor-relationships-service

- missing 'await' keyword in updateFull function
- assigned variable 'relationship' should be 'relationships' in deleteById, causing no-unused-vars and no-undef

- lazy loading not essential now that circular dependencies are eliminated
- call services from globals instead of static variables

This change consolidates object type declarations for Mongoose documents with 'type' properties.

Remove custom errors dictionary from AttackObjectsRepository (no longer in use).

The previous implementation was throwing a bunch of issues. It was easier to just start from 'develop' and and refactor it from scratch.

Identitied four methods implemented in subclasses that were being called from the parent class (BaseService) and migrated them to the parent class.

All 581 regression tests are now passing!

…thods

Converting these two methods to static would require changes throughout the service layer. Rather than exhaust time refactoring them to call the method statically, we should investigate design changes that will organically resolve the issue.